⚠️ No Changeset found

Latest commit: d9be461

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

hmm how about only implementing it only for vite7 then? I forgot what does it do and i remember fighting that filter in one of the e2e tests

This update is also important for compatibility with Vite 8.
I found that starting from version 7.1.0-beta.0, Vite now uses * as the default value for the path during a full-reload.
As a result, the undefined value that the current logic expects is no longer passed in newer versions, which is why full-reload has stopped working in content scripts

You can see the specific change in the Vite source code here:
https://github.com/vitejs/vite/blob/v7.1.0-beta.0/packages/vite/src/node/server/hmr.ts#L712-L717

If there are reasons not to allow all paths, we should at least check for both * and undefined to ensure backward compatibility with older Vite versions while supporting Vite 7.1+ and Vite 8